Skip to content

[Fix] Astra provider CI fails when focused tests are omitted - #1530

Draft
ghost wants to merge 3 commits into
mainfrom
fix/gpt-6-astra-mutation-ci-342bv6tzrna6w
Draft

[Fix] Astra provider CI fails when focused tests are omitted#1530
ghost wants to merge 3 commits into
mainfrom
fix/gpt-6-astra-mutation-ci-342bv6tzrna6w

Conversation

@ghost

@ghost ghost commented Sep 5, 2026

Copy link
Copy Markdown

​Opened on behalf of @navedmerchant. View the task or mention @roomote for follow-up asks.

What changed

  • Keeps hyphen-suffixed direct tests such as openai-native-usage.spec.ts in changed-code mutation runs instead of dropping them when a dot-suffixed direct test also exists.
  • Adds focused decision-table coverage for Astra reasoning normalization, Responses Lite routing, cache behavior, temperature, token fields, and sequential tool calls across the providers merged in [Feat] Add verified GPT-6 Astra support across providers #1506.
  • Simplifies equivalent cache-total and exact-model predicates so mutation testing measures observable behavior rather than module-initialization artifacts.

Why this change was made

The mutation gate on #1506 omitted an existing focused usage suite and left provider guard branches under-specified. The feature was merged while the final CI remediation was still being validated, so these fixes are isolated in this follow-up.

Impact

Changed-code mutation testing now exercises all directly related focused suites and passes for the Astra provider changes. Runtime behavior from #1506 is preserved.

Related PRs

@ghost ghost assigned navedmerchant Sep 5, 2026
@github-actions

github-actions Bot commented Sep 5, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Review status

Thanks for contributing. This comment tracks the review sequence and the next action.

Current step: Resolve the merge conflicts. The review sequence resumes after the branch is mergeable.

Review-state labels are managed by this workflow; do not edit them manually. community-approved is managed the same way — do not add or remove it manually. It signals a fresh community code approval for the current head as an advisory priority only; maintainer review is still required.

This was referenced Sep 5, 2026
Draft
Merged
@codecov

codecov Bot commented Sep 5, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@github-actions github-actions Bot added has-conflicts PR has merge conflicts with the base branch and removed has-conflicts PR has merge conflicts with the base branch labels Sep 6, 2026
@coderabbitai

coderabbitai Bot commented Sep 10, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are limited based on label configuration.

🏷️ Required labels (at least one): 1
  • coderabbit-review-active

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added has-conflicts PR has merge conflicts with the base branch and removed has-conflicts PR has merge conflicts with the base branch labels Sep 10, 2026
@github-actions github-actions Bot added has-conflicts PR has merge conflicts with the base branch and removed has-conflicts PR has merge conflicts with the base branch labels Sep 16, 2026
@github-actions

github-actions Bot commented Oct 24, 2026

Copy link
Copy Markdown
Contributor

This PR has been awaiting author changes for 14 days and will be automatically closed in 7 days. Please address the review comments or leave a comment if you need more time.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@taltas taltas Awaiting requested review from taltas taltas will be requested when the pull request is marked ready for review taltas is a code owner
@JamesRobert20 JamesRobert20 Awaiting requested review from JamesRobert20 JamesRobert20 will be requested when the pull request is marked ready for review JamesRobert20 is a code owner
@edelauna edelauna Awaiting requested review from edelauna edelauna will be requested when the pull request is marked ready for review edelauna is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@navedmerchant navedmerchant

Labels

awaiting-author PR is waiting for the author to address requested changes has-conflicts PR has merge conflicts with the base branch stale-awaiting-author PR is stale while waiting for requested author changes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@navedmerchant